Skip to content

[Domain] Skip evacuated agents in motion updates#237

Merged
learncold merged 1 commit into
mainfrom
codex/skip-evacuated-motion-agents
May 14, 2026
Merged

[Domain] Skip evacuated agents in motion updates#237
learncold merged 1 commit into
mainfrom
codex/skip-evacuated-motion-agents

Conversation

@muzygosu
Copy link
Copy Markdown
Collaborator

Summary

  • Exclude agents that have already reached an exit from the expensive ScenarioSimulationMotionSystem motion hot path.
  • Keep total-agent accounting and simulation completion checks based on the full simulation entity list.
  • Keep the evacuation/motion model unchanged: no exit queueing, flow-capacity model, route-choice changes, or engine API changes.

Related Issue

Area

  • Engine
  • Domain
  • Application
  • Docs
  • Build
  • Analysis
  • Chore

Architecture Check

  • I kept the dependency direction application -> domain -> engine.
  • I did not add Qt UI code to src/domain.
  • I did not add domain or application dependencies to src/engine.
  • I used src/ as the include root.

Verification

  • cmake --preset windows-debug
  • cmake --build --preset build-debug
  • ctest --preset test-debug
  • cmake --build --preset build-engine-domain-debug --target safecrowd_domain -- /m:1 /v:minimal
  • cmake --build --preset build-debug --target safecrowd_app -- /m:1 /v:minimal
  • Not run: ctest was not run for this narrow one-file change; prior full test target is known to have unrelated link blockers around duplicateScenarioDraft / computeScenarioDiffKeys.

Risks / Follow-up

  • This should not change routing or crowd behavior, but it does change which entity list is passed into post-evacuation motion/overlap work.
  • Follow-up performance work should target the Motion algorithm itself if this does not materially improve the 100-agent late-run slowdown.

@learncold learncold force-pushed the codex/skip-evacuated-motion-agents branch from b7c9a2e to 4b7e72b Compare May 14, 2026 05:01
@learncold learncold marked this pull request as ready for review May 14, 2026 05:01
@learncold learncold self-requested a review as a code owner May 14, 2026 05:01
@learncold learncold merged commit ccf0655 into main May 14, 2026
4 checks passed
@learncold learncold deleted the codex/skip-evacuated-motion-agents branch May 14, 2026 05:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Task-대피 완료 에이전트 Motion 계산 제외

2 participants